Skip to content

Fix coco 5302 message can be send multiple times#965

Open
vbillard91 wants to merge 24 commits intodevelop-b2schoolfrom
fix-COCO-5302-message-can-be-send-multiple-times
Open

Fix coco 5302 message can be send multiple times#965
vbillard91 wants to merge 24 commits intodevelop-b2schoolfrom
fix-COCO-5302-message-can-be-send-multiple-times

Conversation

@vbillard91
Copy link
Contributor

@vbillard91 vbillard91 commented Feb 26, 2026

Description

In send message process in conversation, add a function in postgresql to update message status or raise an exception if the message is not in the correct status, to avoid the possibility to send multiple times the same message with different recipient.

Fixes

https://edifice-community.atlassian.net/browse/COCO-5302

Type of change

Please check options that are relevant.

  • Chore (PATCH)
  • Doc (PATCH)
  • Bug fix (PATCH)
  • New feature (MINOR)

Which packages changed?

Please check the name of the package you changed

  • admin
  • app-registry
  • archive
  • auth
  • cas
  • common
  • communication
  • conversation
  • directory
  • feeder
  • infra
  • portal
  • session
  • test
  • tests
  • timeline
  • workspace

Tests

  1. Describe here the tests you performed
  2. Step by step
  3. With expected results

Reminder

  • Security flaws

  • Performance impacts (think bulk !)

  • Unit tests were replayed

  • Unit tests were added and/or changed

  • I have updated the reminder for the version including my modifications

  • All done ! 😃

@vbillard91 vbillard91 force-pushed the fix-COCO-5302-message-can-be-send-multiple-times branch from eebe33e to a7a1921 Compare February 26, 2026 13:27
@vbillard91 vbillard91 requested review from jcbe-ode and pb-jo and removed request for pb-jo February 26, 2026 13:27
@Romu-C Romu-C force-pushed the develop-b2school branch from 5435be5 to c213e67 Compare March 2, 2026 09:50
@jcbe-ode
Copy link
Contributor

jcbe-ode commented Mar 5, 2026

La PR est en conflit, impossible à review

@vbillard91 vbillard91 force-pushed the fix-COCO-5302-message-can-be-send-multiple-times branch from 3f7cd70 to 428c9a4 Compare March 5, 2026 15:57
@vbillard91
Copy link
Contributor Author

La PR est en conflit, impossible à review

c'est corrigé

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants